Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fileIngester #51

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open

fileIngester #51

wants to merge 34 commits into from

Conversation

brian-m-rapp
Copy link
Contributor

  • Added fileIngest module with conditional compilation (--with-ingester arg to configure)
  • Fixed issues reported by Coverity in noaaport tree
  • Increased size of retransmission table arrays to allow for longer latency between time of request and time of reply

brian-m-rapp and others added 30 commits February 27, 2016 12:39
@brian-m-rapp
Added fileIngester tree
fdafc0f
@brian-m-rapp
deleted goesr_lib.h
f8df63d
@brian-m-rapp
Migrated fileIngester to new LDM logging system for ldmProductQueue.c.
1311b0f
@brian-m-rapp
More logging updates.
a7ee031
@brian-m-rapp
Added satellite center position to GOES-R WMO header functions.
39798d2
@brian-m-rapp
Merge https://github.com/Unidata/LDM
348d6fd
@brian-m-rapp
Code clean up.
43d64a1
@brian-m-rapp
More code clean up.
4da1b3a
@brian-m-rapp
Fixed coding problems identified by Coverity.
bd64b94
@brian-m-rapp
DR-18765 - Increase size of retransmission tables to allow for
3aebb8c 
retransmissions up to 1 hour after original broadcast.
@brian-m-rapp
Replaced 'gets' with 'fgets'.
4f71db1
@brian-m-rapp
Added build rule for .hin files to fileIngest/Makefile.am and fixed
6daea0f 
compiler warnings in fileIngester/datelib.c.
@brian-m-rapp
Updated fileIngest/.gitignore
cc4dd82
@brian-m-rapp
Fixed problems from coverity scan.
bf4514a
@brian-m-rapp
Fixed issues reported by coverity scan.
af53393
@brian-m-rapp
Fixed bugs reported by Coverity scan.
e3143ce
@brian-m-rapp
Merge remote-tracking branch 'upstream/master'
20a9a73
@brian-m-rapp
Fixed bug in fileIngest/Makefile.am
17a3f7f
@brian-m-rapp
Add option to pqact to allow WMO header to be stripped from a product.
091977c
@brian-m-rapp
Fixed compiler warnings in filel.c
a7d2d31
Additional changes for header stripping in pqact.
2d97859
Added debug message to filel.c.
e3f653d
@brian-m-rapp
Added additional boundary check to getWmoOffset and restricted WMO
970120b 
search to first 200 bytes of a product in skipWMO.
@brian-m-rapp
Cleanup to skipWMO.
da37f36
Revised placement of skipWMO to allow ftruncate to properly size files.
91a08f6
@brian-m-rapp
Replaced numeric values with named constants in skipWMO().
bbf6151
Additional comments in filel.c
210b6e7
@brian-m-rapp
Added check in stripHeaders() for small product size.
80d49e3
@brian-m-rapp
Fix uninitialized variable in pipe_prodput.
7b00312
Merge remote-tracking branch 'upstream/master'
a0bb98e 
# Conflicts:
#	noaaport/productMaker.c
@brian-m-rapp
Copy link
Contributor Author

NWS updates for:

  • fileIngester program in fileIngest/ sub-directory. Builds with configure --with-ingester.
  • add capability to pqact to strip headers from products using Pattern-Action Rule "-removewmo".

@semmerson
Copy link
Collaborator

Incorporated into next release.

@semmerson
Copy link
Collaborator

Why does fileIngester(1) need to be owned by root and setuid?

@semmerson semmerson reopened this Oct 3, 2016
@brian-m-rapp
Copy link
Contributor Author

Steve,

I wrote that so long ago that my memory is hazy on the subject. I don't
recall if I found it wouldn't work without suid, or if I just did it
because noaaportIngester does it and fileIngester has many of the same
requirements.

Brian

On 10/03/2016 07:58 PM, Steven Emmerson wrote:

Why does fileIngester(1) need to be owned by root and setuid?

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
#51 (comment), or
mute the thread
https://github.com/notifications/unsubscribe-auth/ADr8J43zPixNxd7kdc91hxDS_c1uyPhVks5qwZaagaJpZM4H4DvM.

@CLAassistant
Copy link

CLAassistant commented Jun 28, 2019 •
edited
Loading

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ brapp
❌ brian-m-rapp

brapp seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@brian-m-rapp @semmerson @CLAassistant